Skip to content

Conversation

zachharrison
Copy link
Contributor

Description of the change

Description here

Type of change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation or Development tools (readme, specs, tests, code formatting)

Links

  • Jira issue number: (PUT IT HERE)
  • Process.st launch checklist: (PUT IT HERE)

Checklists

Development

  • Prettier was run (if applicable)
  • The behaviour changes in the pull request are covered by specs
  • All tests related to the changed code pass in development

Paperwork

  • This pull request has a descriptive title and information useful to a reviewer
  • This pull request has a Jira number
  • This pull request has a Process.st launch checklist

Code review

  • Changes have been reviewed by at least one other engineer
  • Security impacts of this change have been considered

@Copilot Copilot AI review requested due to automatic review settings October 9, 2025 21:25
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR appears to be a comprehensive refactoring focused on standardizing styling and theming across the Mint Components library. The main purpose is to implement a unified branding configuration system while improving code organization and consistency.

Key changes include:

  • Implementation of a dynamic branding configuration system with CSS custom properties
  • Consolidation of import statements and file structure reorganization
  • Replacement of hardcoded style values with configurable CSS variables
  • Updates to component styling to use the new theming system

Reviewed Changes

Copilot reviewed 274 out of 282 changed files in this pull request and generated no comments.

Show a summary per file
File Description
packages/stencilbook/src/stencil-storybook.tsx Code formatting changes from spaces to tabs
packages/mint-components/src/utils/utils.ts Added utility function for capitalizing strings with debug logging
packages/mint-components/src/utils/parseStates.ts New utility for parsing state configuration objects
packages/mint-components/src/templates/*.html Updated border styles to use CSS custom properties
packages/mint-components/src/tables/GenericTableView.tsx Replaced hardcoded colors with CSS variables
packages/mint-components/src/stories/PortalTemplates.stories.tsx Reorganized imports and removed readme references
packages/mint-components/src/saasquatch.d.ts Added comprehensive branding configuration interface
packages/mint-components/src/insertfont.ts New utility for dynamic font insertion
packages/mint-components/src/index.html Added branding configuration script and structure
packages/mint-components/src/global/styles.ts Major refactoring to support dynamic branding with CSS custom properties
packages/mint-components/src/global/mixins.ts Updated color references to use new CSS variables
packages/mint-components/src/global/global.ts Added branding configuration initialization logic
packages/mint-components/src/components/views/*.tsx Updated styling to use new theming system
packages/mint-components/src/components/tax-and-cash/**/*.tsx Extensive updates to use new styling system and fix import paths
Files not reviewed (2)
  • packages/bedrock-components/package-lock.json: Language not supported
  • packages/mint-components/package-lock.json: Language not supported
Comments suppressed due to low confidence (5)

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@Copilot Copilot AI review requested due to automatic review settings October 14, 2025 21:02
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 274 out of 282 changed files in this pull request and generated no new comments.

Files not reviewed (2)
  • packages/bedrock-components/package-lock.json: Language not supported
  • packages/mint-components/package-lock.json: Language not supported
Comments suppressed due to low confidence (4)

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@Copilot Copilot AI review requested due to automatic review settings October 15, 2025 21:51
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 274 out of 282 changed files in this pull request and generated no new comments.

Files not reviewed (2)
  • packages/bedrock-components/package-lock.json: Language not supported
  • packages/mint-components/package-lock.json: Language not supported
Comments suppressed due to low confidence (5)

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@Copilot Copilot AI review requested due to automatic review settings October 15, 2025 22:20
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@Copilot Copilot AI review requested due to automatic review settings October 16, 2025 22:22
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 275 out of 283 changed files in this pull request and generated no new comments.

Files not reviewed (2)
  • packages/bedrock-components/package-lock.json: Language not supported
  • packages/mint-components/package-lock.json: Language not supported
Comments suppressed due to low confidence (5)

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@Copilot Copilot AI review requested due to automatic review settings October 17, 2025 21:24
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 275 out of 283 changed files in this pull request and generated no new comments.

Files not reviewed (2)
  • packages/bedrock-components/package-lock.json: Language not supported
  • packages/mint-components/package-lock.json: Language not supported
Comments suppressed due to low confidence (8)

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants